Skip to content

Xiexq0 patch#5672

Closed
xiexq0 wants to merge 5 commits into
NginxProxyManager:developfrom
xiexq0:xiexq0-patch
Closed

Xiexq0 patch#5672
xiexq0 wants to merge 5 commits into
NginxProxyManager:developfrom
xiexq0:xiexq0-patch

Conversation

@xiexq0

@xiexq0 xiexq0 commented Jun 24, 2026

Copy link
Copy Markdown

Why

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • API changes
  • Performance improvement
  • Test addition or update

AI Usage

  • AI was used to write this
  • AI was used to review this

@nginxproxymanagerci

Copy link
Copy Markdown

Docker Image for build 3 is available on DockerHub:

nginxproxymanager/nginx-proxy-manager-dev:pr-5672

Note

Ensure you backup your NPM instance before testing this image! Especially if there are database changes.
This is a different docker image namespace than the official image.

Warning

Changes and additions to DNS Providers require verification by at least 2 members of the community!

@toviszsolt

Copy link
Copy Markdown
Contributor

@xiexq0
These changes are unwanted. The PR type you selected (non-breaking change that adds functionality) is completely incorrect, as this PR does not include a new feature. Instead, it breaks the Docker image by modifying the ports, which can already be mapped however needed.

@xiexq0 xiexq0 closed this by deleting the head repository Jun 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants